Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iotjs: Port to alt runtime using async-lite #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Nov 9, 2018

i2c is not needed because native I2C module will be used

Usage:

git clone https://github.com/rzr/async-lite iotjs_modules/async
iotjs test
{
  "pressure": 99406.04496626806,
  "temperature": 22.5
}

Change-Id: I97d4248d87f59cd9468dc7fb90c1f2c04dd1d55c
Signed-off-by: Philippe Coval philippe.coval@osg.samsung.com

rzr added a commit to TizenTeam/bmp085-sensor that referenced this pull request Nov 9, 2018
i2c is not needed because native I2C module will be used

Usage:

```
git clone https://github.com/rzr/async-lite iotjs_modules/async
iotjs test
{
  "pressure": 99406.04496626806,
  "temperature": 22.5
}
```

Forwarded: dbridges#7
Change-Id: I97d4248d87f59cd9468dc7fb90c1f2c04dd1d55c
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
@rzr
Copy link
Contributor Author

rzr commented Nov 20, 2018

Relate-to: https://www.slideshare.net/rzrfreefr/webthingiotjstizenrtcdl201820181117rzr/24

@rzr
Copy link
Contributor Author

rzr commented Dec 20, 2018

Please review, without feedback I'll consider to fork it into:
https://www.npmjs.com/org/abandonware

@rzr
Copy link
Contributor Author

rzr commented Feb 25, 2019

Is this project still maintained ?

A new version in npm will this change will be welcome

Then I will update downstream project:

https://github.com/rzr/mozilla-iot-generic-sensors-adapter

rzr added a commit to rzr/generic-sensors-lite that referenced this pull request Feb 27, 2019
Change-Id: I8f9811e16d6049b5b2a344e21d25249c68420088
Relate-to: dbridges/bmp085-sensor#7
Signed-off-by: Philippe Coval <p.coval@samsung.com>
@rzr
Copy link
Contributor Author

rzr commented Apr 17, 2019

It would help to have this change in released version (git tag + npm),
then I'll pin the version in generic-sensors-lite:

https://libraries.io/npm/generic-sensors-lite

@rzr
Copy link
Contributor Author

rzr commented Jun 28, 2019

Hi, current release 0.0.5 is 10 months old,
is a new one planned ? if yes please consider this patch

If not I would like to help in co-maintenance.

Thanks

@rzr
Copy link
Contributor Author

rzr commented Jul 15, 2019

Until I get feedback from maintainer, I will use this fork:
https://github.com/abandonware/bmp085-sensor

@rzr rzr force-pushed the sandbox/rzr/review/master branch from 0e8fdca to b37e35c Compare July 15, 2019 11:44
i2c is not needed because native I2C module will be used

Usage with IoT.js:

```
git clone https://github.com/rzr/iotjs-async iotjs_modules/async
iotjs test
{
  "pressure": 99406.04496626806,
  "temperature": 22.5
}
```

Change-Id: I97d4248d87f59cd9468dc7fb90c1f2c04dd1d55c
Forwarded: dbridges#7
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
@rzr rzr force-pushed the sandbox/rzr/review/master branch from b37e35c to 7f7d5ef Compare July 15, 2019 11:54
@rzr
Copy link
Contributor Author

rzr commented Aug 28, 2019

This change has been released in:
https://www.npmjs.com/package/@abandonware/bmp085-sensor/v/0.0.6-1

If merged I will rebase on master

@rzr
Copy link
Contributor Author

rzr commented Mar 7, 2020

Please merge I will rebase then for next version

Current is:
https://www.npmjs.com/package/@abandonware/bmp085-sensor/v/0.0.6-2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant